Skip to content

feat(textures): author animated azure water frame strips and manifests (#68) - #256

Merged
BharathASL merged 1 commit into
mainfrom
address_issue_sixty_eight
Nov 4, 2026
Merged

BharathASL merged 1 commit into
mainfrom
address_issue_sixty_eight

Conversation

@BharathASL

Copy link
Copy Markdown
Contributor

Description

Closes #68.

Implements Task 2.1.9 by authoring animated still and flowing water textures matching the clean, flat trailer aesthetic seen in official Minecraft animations and promotional shorts.

  • water_still: Authored as a uniform, flat translucent fluid (#ffffff, opacity 0.78) across a 16-frame animation strip. Eliminates internal grid lines and wireframe artifacts on 3D cube block faces under water, providing crystal-clear visibility and pure trailer azure when multiplied by dynamic engine biome colormaps.
  • water_flow: Authored with delicate longitudinal current ribbons and downstream wave crests to indicate flow momentum without harsh crosshairs.
  • Animation Manifests: Paired with companion manifests (water_still.png.mcmeta with frametime 2 and interpolation, and water_flow.png.mcmeta with frametime 1 and interpolation).
  • Toolchain Updates:
    • tools/lib/vector-linter.mjs: Updated viewBox validation to support multi-frame vertical animation strips (0 0 512 (512 * N)).
    • tools/test-tiling.mjs: Evaluates vertical seam continuity per frame for multi-frame strips. Both textures achieve zero discontinuity.
    • tools/build.mjs & tools/release.py: Excluded fluid textures (water_*, lava_*) from LabPBR normal and specular map generation and release inventory requirements.

Affected Assets

  • textures/block/water_still.svg
  • textures/block/water_still.png.mcmeta
  • textures/block/water_flow.svg
  • textures/block/water_flow.png.mcmeta
  • tools/lib/vector-linter.mjs
  • tools/test/vector-linter.test.mjs
  • tools/test-tiling.mjs
  • tools/build.mjs
  • tools/release.py
  • .gitignore

Type of Change

  • feat: New vector texture master, blockstate, or compiler capability
  • fix: Tiling fix, palette correction, or bugfix
  • docs: Documentation improvement
  • chore / refactor: Maintenance, dependencies, or codebase cleanup

Contributor Checklist

Please check all applicable boxes before requesting review:

Vector Texture Standards (if adding/modifying SVGs)

  • Authored as a clean $512\times512$ SVG (viewBox="0 0 512 512" or vertical multi-frame strip 0 0 512 8192).
  • Primary shapes are aligned to the $32\text{px}$ texel grid ($16\times16$ grid).
  • Seamless Toroidal Tiling: Checked and verified that elements wrapping across $X$ and $Y$ edges align perfectly without seams or chopped shapes.
  • Ore Consistency: If adding an ore texture, the stone background geometry and corner radius (rx) match textures/block/stone.svg exactly.
  • Free of leftover AI generation comments, unnecessary editor namespaces, or embedded raster images.

Build & Verification

  • Ran npm run build locally and verified that the pack compiles successfully.
  • Verified textures in-game or inspected the rasterized PNG outputs in dist/.

Git Hygiene & Standards

  • Commit message(s) follow Conventional Commits (e.g., feat(textures): ...).
  • Every commit is signed off with the Developer Certificate of Origin (git commit -s).
  • Branch is rebased cleanly onto latest main with no merge commits.
  • Formatting complies with .editorconfig (2-space indent, LF endings, trailing newline).

#68)

Add still and flowing water vector masters, companion animation manifests, multi-frame viewBox support, and fluid PBR exemptions.

Signed-off-by: Bharath Sendhurpandi <bharathasl74185@gmail.com>
@BharathASL
BharathASL added this pull request to the merge queue Nov 4, 2026
Merged via the queue into main with commit eb794d3 Nov 4, 2026
3 checks passed
@BharathASL
BharathASL deleted the address_issue_sixty_eight branch November 4, 2026 14:24

@ninja6-agent ninja6-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task 2.1.9: Core Fluids: Animated Azure Water (Frame Strip & .mcmeta)

1 participant